Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GH4331/GH4349: Migrate GHA Artifact to V4 API #4350

Merged
merged 2 commits into from
Oct 4, 2024

Conversation

@devlead devlead marked this pull request as ready for review September 30, 2024 20:44
@devlead devlead requested a review from a team September 30, 2024 20:45
@devlead
Copy link
Member Author

devlead commented Sep 30, 2024

@cake-build/cake-team this should be ready for review, integration tests no longer cause warnings like this
image
Example previous run: https://github.com/cake-build/cake/actions/runs/11092875166

Upload / Download artifact is completely rewritten for the new v4 APIs but public Cake Common APIs/Aliases remain the same and integration tests look to generate same set of artifacts

Previous: https://github.com/cake-build/cake/actions/runs/11092875166#artifacts
Now: https://github.com/cake-build/cake/actions/runs/11113486680?pr=4350#artifacts

The sizes are now correct previously it reported the uncompressed size, and uploads are noticeably faster.

@devlead devlead requested a review from gep13 October 4, 2024 10:53
@devlead devlead enabled auto-merge October 4, 2024 11:42
Copy link
Member

@gep13 gep13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@devlead devlead merged commit 86ad011 into cake-build:develop Oct 4, 2024
15 checks passed
@gep13
Copy link
Member

gep13 commented Oct 4, 2024

@devlead thank you for getting this done!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants